Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Red Hat family distro module and Fedora distro module #252

Merged
merged 1 commit into from
Jun 4, 2020

Conversation

Conan-Kudo
Copy link
Contributor

This change adds support for testing Red Hat based distributions, and adds a Fedora module for testing Fedora cloud images.

This PR is the beginning of investigating using img_proof for automatically testing Fedora Cloud Edition images by the Fedora Cloud SIG.

(cc: @dustymabe, @major)

@Conan-Kudo Conan-Kudo force-pushed the add-fedora branch 2 times, most recently from 9cf4a64 to 6688255 Compare June 3, 2020 19:50
@smarlowucf
Copy link
Collaborator

smarlowucf commented Jun 3, 2020

Hi @Conan-Kudo, thanks for the contributions 👍 . One last piece to update is the set_distro method which gets the class based on the distro_name argument. https://github.com/SUSE-Enceladus/img-proof/blob/master/img_proof/ipa_cloud.py#L377 As a note the distro_name provided is converted to lowercase prior to comparison.

Also, all commits should be signed. I have some information on getting a signing key setup in a gist https://gist.github.com/smarlowucf/25dbd6021da1a8bdece248fa8d7ece71#create-pgp-key-for-signing-commits-and-add-public-key-to-github-account. Hope this helps.

This change adds support for testing Red Hat based distributions,
and adds a Fedora module for testing Fedora cloud images.

Signed-off-by: Neal Gompa <[email protected]>
@Conan-Kudo
Copy link
Contributor Author

@smarlowucf I'm not entirely sure why you force GPG signed commits (they're not particularly useful in this model of contributions), but here you go, signed commits...

@smarlowucf smarlowucf merged commit cbb41c9 into SUSE-Enceladus:master Jun 4, 2020
@smarlowucf
Copy link
Collaborator

Looks good, thanks 👍 .

@Conan-Kudo Conan-Kudo deleted the add-fedora branch June 4, 2020 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants